Refactor jabrefHost.py for future implement#14825
Refactor jabrefHost.py for future implement#14825sonthien22501 wants to merge 1 commit intoJabRef:mainfrom
Conversation
|
For the connection between Jabref and Plugin extension, i have seen that Jabref its own has that feature. |
|
Closing this PR as it was opened against the wrong repository by mistake. I intended to open this on my fork. |
Please try to use http only - this should reduce complexity. - Read on from https://github.com/JabRef/jabref/blob/main/jabsrv/src/main/java/org/jabref/http/server/command/SelectEntriesCommand.java to learn about the command resource @palukku I think, this is our intended way, isn't it? |
|
Note that your PR will not be reviewed/accepted until you have gone through the mandatory checks in the description and marked each of them them exactly in the format of |
|
This also refs #14823 |
Closes _____
Problem
The original jabrefHost.py script was hardcoded to look for JabRef in the standard macOS installation folder (/Applications/JabRef.app). Since I am developing and running JabRef from the source code (using Gradle), I don't have a packaged app installed there. This caused the Native Messaging host to fail because it couldn't find the executable.
Solution
I modified jabrefHost.py to add a development path check. The script now looks for the executable in two places:
1. First: It checks my local build folder (jabgui/build/install/jabgui/bin/JabRef).
2. Second: It falls back to the standard system path
Result
This allows us to test the browser extension against the latest code changes I make in the IDE immediately, without needing to package and install the full application every time I change a line of code. It streamlines the development loop significantly.
Steps to test
Mandatory checks
CHANGELOG.mdin a way that is understandable for the average user (if change is visible to the user)